[l10n] Updated Italian translation.

x11: Add/Fix '(type *)' g-i annotations

Simplify GtkActionHelper

Now that the last GMenuModel-based user of GtkActionHelper is gone, we
can substantially simplify it.

https://bugzilla.gnome.org/show_bug.cgi?id=721836

Use GtkMenuTracker for Quartz backend.

https://bugzilla.gnome.org/show_bug.cgi?id=710351

Move get_key_equivalent() to gdk quartz utils.

https://bugzilla.gnome.org/show_bug.cgi?id=710351

Updated German translation

GtkMenuTracker: fix hidden-when='' vs. separators

Ensure that adding hidden-when='' to a menu item does not produce an
extra separator item as a side effect.

https://bugzilla.gnome.org/show_bug.cgi?id=688421

bloatpad: test hidden-when=''

Cook up some silly cases to test out the hidden-when='' attribute.

 - make sure hidden-when='action-missing' shows/hides items based on
   actions being created and destroyed

 - make sure hidden-when='action-disabled' shows/hides items based on
   actions being enabled and disabled

 - make sure hidden-when='action-missing' doesn't hide items when the
   action is merely disabled

https://bugzilla.gnome.org/show_bug.cgi?id=688421

GtkMenuTracker: remove hidden items from the menu

Modify the tracker so that it manages the visibility of
GtkMenuTrackerItem by issuing insert and remove callbacks to the
user of the tracker.

This works by treating the GtkMenuTrackerItem as a virtual section which
contains 1 item when the item is visible and 0 items when it is hidden.

For efficiency reasons, we only employ this trick in the case that the
item has a hidden-when='' attribute set on it.

https://bugzilla.gnome.org/show_bug.cgi?id=688421

GtkMenuTrackerItem: add an internal 'visible' flag

Add an internal API for checking if a GtkMenuTrackerItem is visible,
along with a signal for reporting changes in that flag.  The item will
become invisible in situations according to the new hidden-when=''
attribute, which can be set to 'action-disabled' or 'action-missing'.

This new flag doesn't actually do anything yet, and none of the
consumers of GtkMenuTracker do anything with it (nor should they).  A
followup patch will address the issue.

https://bugzilla.gnome.org/show_bug.cgi?id=688421

GtkMenuTracker: rework action removal a bit

Refactor the code in the action observer remove function in order to
make way for the (efficient) handling of hiding of the item in the case
that hidden-when='' is given.

https://bugzilla.gnome.org/show_bug.cgi?id=688421

GtkMenuTrackerItem: small logic tweak

Strictly speaking, can_activate should always be set back to FALSE when
the action disappears from the muxer (since we can't activate it
anymore) but we forgot to do that.

This 'bug' could never cause a problem because 'can_activate' is never
directly queried for anything at all and the item would get marked
insensitive anyway.  As soon as the action was re-added, can_activate
would be recalculated based on the new action before anything else could
happen.

All the same, this should be cleared here.

https://bugzilla.gnome.org/show_bug.cgi?id=688421

GtkMenuTracker: cache result of hash lookup

Remove a hash lookup from the separator sync logic (which is run every
time we change a menu).  Instead, we do the lookup when creating the
section and cache the result.

This refactor will also help us in a future commit to add support for
hiding menu items based on missing actions.

https://bugzilla.gnome.org/show_bug.cgi?id=688421

colorbutton: fix crash when destroying the colorbutton from the color-set signal

Updated Spanish translation

PixelCache: Ensure clean cairo_t state in draw

This adds save/restore calls to the clear-to-transparent call in
the pixel cache, to avoid changing the default color of the
cairo_t. It also removes a call set_operator call that is no longer
necessary (it was trying to manually restore the state).

https://bugzilla.gnome.org/show_bug.cgi?id=721480

fix prototypes of signal callbacks in the test suite

The signal callbacks are defined to take pointers as their arguments, but the
callbacks found in testsuite/gtk/builder.c are passing a GParamSpec by value
as the second argument.  This confuses and angers the compiler on ppc64el,
resulting in segfaults after return from the function due to stack-smashing
by the (completely-unused) argument.

https://bugzilla.gnome.org/show_bug.cgi?id=721700

Add icon menu to bloatpad.

https://bugzilla.gnome.org/show_bug.cgi?id=710351

GtkIconInfo: add gtk_icon_info_is_symbolic()

https://bugzilla.gnome.org/show_bug.cgi?id=710351

gtkapplication-quartz: clean up inhibit code

When testing with bloatpad, the existing inhibit code seems not to be
working at all.  Replace it with a cleaner and simpler version that
works.

https://bugzilla.gnome.org/show_bug.cgi?id=720551

app window: test actiongroup across destroy

Make sure that we don't violate the interface contract of GActionGroup
just because gtk_widget_destroy() was called.

https://bugzilla.gnome.org/show_bug.cgi?id=710351

Fix GtkApplicationWindow action group implementation

GtkApplicationWindow frees its internal action group on dispose for the
usual reasons: to avoid the possibility of reference cycles caused by
actions referring back to the window again.

Unfortunately, if it happens to be inside of a GtkActionMuxer at the
time that it is disposed, it will (eventually) be removed from the muxer
after it has been disposed.  Removing an action group from a muxer
involves a call to g_action_group_list_actions() which will crash
because the internal action group to which we normally delegate the call
has been freed.

A future patch that reworks the quartz menu code will introduce a use of
GtkActionMuxer in a way that causes exactly this problem.

We can guard against the problem in a number of ways.

First, we can avoid the entire situation by ensuring that we are removed
from the muxer before we destroy the action group.  To this end, we
delay destruction of the action group until after the chain-up to the
dispose of GtkWindow (which is where the window is removed from the
GtkApplication).

Secondly, we can add checks to each of our GActionGroup and GActionMap
implementation functions to check that the internal action group is
still alive before we attempt to delegate to it.

We have to be careful, though: because our _list_actions() call will
suddenly be returning an empty list, people watching the group from
outside will have expected to see "action-removed" calls for the
now-missing items.  Make sure we send those. but only if someone is
watching.

https://bugzilla.gnome.org/show_bug.cgi?id=710351

Add a reftest for GtkActionBar child ordering

This shows that GtkActionBar and GtkBox treat pack-start/-end
the same.

Updated POTFILES.in

GtkActionBar: Just leave the center widget as NULL, initially

No need to construct a box that gets thrown away later.
Plus, this fixes the defaultvalue test for GtkActionBar.

Release-note the headerbar child order change

Make testsplitheaders more robust

Same empty string issue that I fixed in GtkHeaderBar a few days
ago.

Updated Hungarian properties translation

Add an action bar test

https://bugzilla.gnome.org/show_bug.cgi?id=721665

Fix order of pack-end widgets

https://bugzilla.gnome.org/show_bug.cgi?id=721665

widget-factory: add actionbar

https://bugzilla.gnome.org/show_bug.cgi?id=721665

tests: use actionbar in headerbar test

This is a great example of where headerbar should not be used.

https://bugzilla.gnome.org/show_bug.cgi?id=721665

Add GtkActionBar

A widget intended to offer contextual actions for a given view.
It allows packing children into the start or end as well as offering
a single centered child box.

https://bugzilla.gnome.org/show_bug.cgi?id=721665

Updated Spanish translation

Updated Spanish translation

GtkHeaderBar: Fix positioning of pack-end widgets

This makes the positioning of pack-start and pack-end
widgets symmetric.

Make testtitlebar more versatile

Test adding multiple custom widgets at each end. This exposes
that we're doing it wrong - the pack-end widgets are ordered
in the wrong way.

Update Kazakh translation

Avoid compiler warnings

filechooser: Document the settings XML

Be more careful when parsing gtk-decoration-layout

When the string is empty, breaking it at : will not give
us two tokens, and we were parsing garbage, leading to crash.
https://bugzilla.gnome.org/show_bug.cgi?id=721635

quartz: implement gdk_window_set_shadow_width()

Use the information to allow dragging windows all the way to the top of
the screen (ie: allow the top shadow to go under the menubar).

https://bugzilla.gnome.org/show_bug.cgi?id=720374

quartz: fix manual window move

We need to have gdk skip standard processing of events when we are in
manual move in addition to manual resize.

https://bugzilla.gnome.org/show_bug.cgi?id=720357

tests: add menu alignments to the menubutton test

https://bugzilla.gnome.org/show_bug.cgi?id=720939

tests: add some space to menubutton test

https://bugzilla.gnome.org/show_bug.cgi?id=720939

Fix positioning of up menu button popup

Use the menu allocation instead of the request size.

https://bugzilla.gnome.org/show_bug.cgi?id=720939

bloatpad: fix the action associated with the notification

Notifications can only be associated with application actions,
but clear is a window action. Introduce a "clear-all" action
that forwards to clear on all windows.

https://bugzilla.gnome.org/show_bug.cgi?id=721633

Add an example for split headers

This shows how to deal with variability in window decorations
with split header bars.

GtkHeaderBar: Refresh window buttons when settings change

We already do this when the shell-shows-menu settings changes,
but we forgot to do the same for the decoration layout.

Tajik translation updated

[l10n] Updated Italian translation.

GtkMenuItem: fix initial accessible selection state

When the accessible is created for an already-selected
menuitem, we were not properly setting the state of the
the accessible. Fix that.

x11: Include missing public headers to GdkX11 gir build

Add introspection friendly version of gtk_tree_path_new_from_indices

Add gtk_tree_path_new_from_indicesv which takes an array of
integers with a length. Use "Rename to" annotation to rename the
method as gtk_tree_path_new_from_indices. This is needed because
the original method takes variadic arguments which is not supported
by introspection.

https://bugzilla.gnome.org/show_bug.cgi?id=706119

Updated Brazilian Portuguese translation for UI

Updated Brazilian Portuguese translation for Properties

a11y: Fix keybindings in menus

GtkMenuItemAccessible was assuming that an accel label is
always the immediate child of a menu item. It also did not
deal with manually set accels.  Fix both of these.
https://bugzilla.gnome.org/show_bug.cgi?id=721086

GtkAccelLabel: Add a setter for manual accels

This is needed to keep the a11y keybinding code working.

Emit a11y state change notification for color swatches

Improve the color swatch accessible to set a proper role
depending on whether the swatch is selectable or not, and
set the checked atk state when appropriate.

https://bugzilla.gnome.org/show_bug.cgi?id=721048

Color chooser: More accessible names

Set the accessible name of the 'custom color' button too.

Update a11y tests to match the last change

Color swatches have names now, not descriptions.

Nicer nicks for GtkMenuButton properties

Requested in
https://bugzilla.gnome.org/show_bug.cgi?id=720945

GtkColorChooserWidget: Set accessible names

As pointed out in
https://bugzilla.gnome.org/show_bug.cgi?id=721053
we should set the accessible name on the color swatch widgets,
not the descriptions.

GtkModelMenuItem: set initial accessible role

The initial state of GtkModelMenuItem is that of a normal menu
item, but the accessible we are using, GtkCheckMenuItemAccessible,
starts out with a role of 'check menu item'. Fix that up by
explicitly setting the initial accessible role.

Drop custom colors from a11y output

These were inadvertedly included in the previous commit, but
no custom colors are present in the gnome-continuous test sessions.

GtkCheckMenuItemAccessible: Make work with model menu items

GtkModelMenuItem does not emit the ::toggled signal when a radio
item is activated, so listen for property notification for that
property. We still keep the ::toggled signal handler, in order
to not break other uses of check and radio menu items.

https://bugzilla.gnome.org/show_bug.cgi?id=720983

GtkModelMenuItem: Add some property notification

GtkModelMenuItem emits no property notification, since none of its
properties are readable. But the toggled property is just a proxy
for GtkCheckMenuItem::active, so we should ensure that property
notification is emitted for the ::active property.

bgo #721407 - Fix scrolled window to work with template subclasses.

Ensure the hscrollbar & vscrollbar at gtk_scrolled_window_add() time,
this allows one to subclass GtkScrolledWindow with templates and add
children, as this will happen at instance initialization time before
the construct adjustment properties take effect.

GtkColorSwatch: Set an accessible role

And update the colorchooser a11y test to a) use GtkColorChooser
instead of the deprecated GtkColorSelection and b) match this
change. Pointed out in
https://bugzilla.gnome.org/show_bug.cgi?id=721045

testbbox: Remove unused variable

This was the only compiler warning in Continuous for gtk3.

GtkInfoBar: Set a more accurate accessible role

And update the a11y tests to match. Pointed out in
https://bugzilla.gnome.org/show_bug.cgi?id=721042

GtkDrawingArea: Set an accessible role

And update the a11y tests to match. Pointed out in
https://bugzilla.gnome.org/show_bug.cgi?id=721040

Trivial formatting fix

Updated Czech translation

Deal with broken _NET_WORKAREA more gracefully

fvwm seems to have problems keeping _NET_WORKAREA in sync with
the number of desktops. Instead of reading garbage, silently use
the full screen as workarea for desktops that are not covered
by the _NET_WORKAREA property.

https://bugzilla.gnome.org/show_bug.cgi?id=698248

x11: Avoid a possible memory leak in an error case

If _NET_WORKAREA is set to nonsensical values, we should still
free the memory before returning.

Trivial typo fix

gtk: remove unused GTK_BUTTONBOX_DEFAULT_STYLE from gtkenums.h tests: remove GTK_BUTTONBOX_DEFAULT_STYLE from testbbox.c and fix crashing

https://bugzilla.gnome.org/show_bug.cgi?id=721317

GtkNotebook: Fix a problem with tab states

When replacing the tab label, we were not adding the
active-page style class. Fix that.

Raleigh: Improve notebook tab theming

The theme was not kept up-to-date when notebook tabs were
recently refactored. Fix it to show some indication of the
active tab.

Raleigh: Improve scale theming

Add some visual indication for fine adjustment mode in scales
and scrollbars.

Raleigh: Improve titlebar appearance

The default theme wasn't kept up-to-date as the header bar
and titlebar theming was tweaked. Update it to look non-broken,
at least.

GtkAboutDialog: be consistent about link activation

As pointed out in
https://bugzilla.gnome.org/show_bug.cgi?id=721394,
links in the credits part were not going through the
::link-activated signal. Fix that.

Trivial grammar fix

GtkLabelAccessible: reorganize the code

Separate the GtkLabelAccessible, GtkLabelAccessibleLink and
GtkLabelAccessibleLinkImpl implementations more clearly.

GtkLabelAccessible: update state of links

This commit makes the LinkImpl children of label accessibles
inherit all their state, except for focused and visited, which
are handled directly.

https://bugzilla.gnome.org/show_bug.cgi?id=721406

a11y tests: Add a test for multi-link labels

GtkLabelAccessible: Implement AtkHypertext

This commit makes the label accessible implement AtkHypertext,
which returns a AtkHyperlink object for each link in the text.
At the same time, add AtkHyperlinkImpl objects as children
to the label accessible.

Also some private API to indicate that links have changed, and
call that from GtkLabel when needed.

Adjust expected output of the affected a11y tests.

https://bugzilla.gnome.org/show_bug.cgi?id=721410
https://bugzilla.gnome.org/show_bug.cgi?id=721421

GtkLabel: Add private api for links

This exposes enough information to implement the AtkHyperText
interface.

More private link api

Add a gtklabelprivate.h header

For now, move the few private functions from gtklabel.h here.

a11y tests: Prepare to dump AtkHypertext

This will be used for links in labels soon.
As a side-effect, the output for AtkHyperlinkImpl changes
slightly. Update expected output accordingly.

GtkLinkButtonAccessible: fix broken AtkHyperlink implementation

The get_end_index implementation was casting the accessible
to AtkText even though GtkLinkButtonAccessible does not
implement this interface. This did not show up in the a11y
tests because the they were not dumping the affected AtkHyperlinkImpl
properties. Oops.

Deprecated the decoration-button-layout style property

This is now done via a setting + regular property, for
better control, and the style property is ignored.

Update Chinese simplified translation

GtkMenuTracker: tweak separator logic

Ignacio Casal Quinteiro reported a problem whereby an empty section at
the start of a menu has a separator placed after it.  This was caused by
the implementation of the logic that separators should be inserted at
the top of all non-empty sections that are not the first section.  This
logic is obviously incorrect in the case that the first section is empty
(in which case we would not expect to see a separator at the top of the
second section).

Change the logic so that we only insert separators when we see a
non-zero number of actual items in the menu before us.

https://bugzilla.gnome.org/show_bug.cgi?id=721119

Updated Lithuanian translation

Updated Brazilian Portuguese translation for UI

Updated Brazilian Portuguese translation for Properties

configure: use CFLAGS, etc. _FOR_BUILD as well

When not doing cross-builds, use the values of CFLAGS, CPPFLAGS and
LDFLAGS as the default value for CFLAGS_FOR_BUILD, CPPFLAGS_FOR_BUILD
and LDFLAGS_FOR_BUILD, respectively.

This avoids having to manually specify these variables in order to get
extract-strings to build properly.

This should really be handled by ax_prog_cc_for_build.m4.  That has been
reported upstream.  This is a workaround for now.

https://bugzilla.gnome.org/show_bug.cgi?id=721346

window: Document GtkWindow::set-focus

Not sure why this was never documented.